Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Enterprise admonitions #1215

Merged
merged 8 commits into from
Feb 2, 2024
Merged

Update Enterprise admonitions #1215

merged 8 commits into from
Feb 2, 2024

Conversation

ZPain8464
Copy link
Contributor

@ZPain8464 ZPain8464 commented Feb 1, 2024

This PR mainly updates old admonitions, but it also adds a few:

  • Tooljet guide
  • Jenkins guide
  • Integrations page
  • Authorization
  • Authentication
  • Device identity

I should add this is just the initial use case for the admonition. I'd like to strategically place it with more compelling text to help broadcast Enterprise features and use cases.

Resolves https://github.com/pomerium/internal/issues/1711

Copy link

netlify bot commented Feb 1, 2024

Deploy Preview for pomerium-docs ready!

Name Link
🔨 Latest commit c10d5b2
🔍 Latest deploy log https://app.netlify.com/sites/pomerium-docs/deploys/65bcf9b426200900089145b7
😎 Deploy Preview https://deploy-preview-1215--pomerium-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -54,6 +54,10 @@ By configuring your applications to route requests to Pomerium’s Proxy service

## External data sources (Enterprise)

:::enterprise Pomerium Enterprise
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

side note: now that we have a custom component for this admonition, we may want to implement a default title in it, so we wouldn't need to repeat "Pomerium Enterprise" each time we use it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on their example, the title must be manually entered: https://docusaurus.io/docs/markdown-features/admonitions#custom-admonition-type-components. If it's possible to add a default title, we can do that for sure.

@ZPain8464 ZPain8464 merged commit 6c6bfd9 into main Feb 2, 2024
4 checks passed
@ZPain8464 ZPain8464 deleted the zpain/update-admonitions branch February 2, 2024 14:21
backport-actions-token bot pushed a commit that referenced this pull request Feb 2, 2024
* adds ent admonition to authentication page

* adds ent admonition to authorize

* adds ent admonition to device identity section

* adds ent admonition to integrations page

* adds ent admonitions to two guides

* fixes incorrect link

* fixes cspell typo

* Update content/docs/capabilities/authorization.mdx

Co-authored-by: Kenneth Jenkins <[email protected]>

---------

Co-authored-by: Kenneth Jenkins <[email protected]>
ZPain8464 added a commit that referenced this pull request Feb 2, 2024
Update Enterprise admonitions (#1215)

* adds ent admonition to authentication page

* adds ent admonition to authorize

* adds ent admonition to device identity section

* adds ent admonition to integrations page

* adds ent admonitions to two guides

* fixes incorrect link

* fixes cspell typo

* Update content/docs/capabilities/authorization.mdx



---------

Co-authored-by: zachary painter <[email protected]>
Co-authored-by: Kenneth Jenkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants